添加安装阶段对整合包可选文件的支持 by CaveNightingale · Pull Request #1771 · HMCL-dev/HMCL

CaveNightingale · 2022-10-03T11:23:30Z

动机

我们目前在运营一个服务器，服务器提供两组模组，一组要求玩家必须安装，例如匠神、农夫乐事等，另一组推荐玩家选择安装，例如投影图、FreeCam等。

我们曾经尝试使用整合包格式分发，事实上Modrinth和CurseForge整合包格式均支持声明可选模组/文件，但发现HMCL会直接将这些文件当作必选处理。

目前，我们仍然在使用直接压缩.jar文件的方式分发模组。我们希望未来能够通过分发HMCL整合包的形式分发我们的服务器客户端。

描述

识别Modrinth和CurseForge格式中的可选文件，导入向导中增加选择可选文件的步骤页面。
在导入时，有可选文件的整合包将会在安装页面有一个按钮选择可选文件，用户也可以点击原有的安装按键跳过选择可选文件阶段直接全量安装。
在可选文件选择页面，提供一个类似模组管理页面的列表，显示文件名称和模组名称（联网查找），左侧有勾选框，选中代表安装，默认全部选中。
在选择页面底部，提供一个安装按键，点击以完成安装，如果用户选择返回上一页，在可选文件页面的勾选状态也保留。
不改变不含可选文件或者非Modrinth和CurseForge格式的整合包的安装流程，在导入页面不出现进入选择可选文件步骤的按钮。

风险

增加导入带有可选文件的整合包时的网络流量。

非目标

实现导入后变更安装文件的功能，这可能是下一步要做的事情。
更改从URL导入时的行为。

burningtnt · 2023-05-23T12:59:03Z

Merge Conflict 了，记得改一改

zkitefly · 2023-07-29T14:18:26Z

ping @CaveNightingale
麻烦把这个 PR 更新一下

burningtnt · 2023-07-29T14:21:07Z

人已经 9 个月不见了，大概是跑了（

CaveNightingale · 2023-07-29T17:09:53Z

ping @CaveNightingale 麻烦把这个 PR 更新一下

好的

hejiehao · 2023-07-30T02:37:42Z

《跑了》

CaveNightingale · 2023-07-30T08:48:05Z

完成

zkitefly · 2023-08-07T10:41:37Z

@huanghongxun 这个已经一年没合了

zkitefly · 2023-12-31T00:28:38Z

功能请求

要不加个全选（或全不选）按钮？

zkitefly · 2023-12-31T00:35:10Z

Curse 整合包导入测试

注：未测试安装

RLCraft 1.12.2 - Release v2.9.3(改造版).zip

HMCL-3.5.SNAPSHOT(需要将zip后缀改为jar).zip

问题：若网络环境不佳，Curse 整合包下面的文件名显示是直接为空白，我觉得这个文件名获取可能会有点问题？

zkitefly · 2023-12-31T01:08:32Z

Modrinth 整合包导入测试

注：未测试安装

Cobblemon Modpack [Fabric] 1.4.1(改造版).zip

HMCL-3.5.SNAPSHOT(需要将zip后缀改为jar).zip

建议：我发现下方的文件选择没标题可能会让用户不知道是什么东西，我建议在上面加个标题

zkitefly · 2023-12-31T01:55:14Z

Modrinth 整合包安装测试

files 列表：

fancymenu_fabric_2.14.10-2_MC_1.20.1.jar # 可选
lazydfu-0.1.3.jar # 必选
notenoughanimations-fabric-1.6.4-mc1.20.jar # 必选
krypton-0.2.3.jar # 必选
Xaeros_Minimap_23.9.3_Fabric_1.20.jar # 可选
cloth-config-11.1.118-fabric.jar # 可选

Cobblemon Modpack [Fabric] 1.4.1(改造＋精简).zip

HMCL-3.5.SNAPSHOT(需要将zip后缀改为jar).zip

burningtnt · 2023-12-31T01:55:46Z

请将加载整合包文件的 Task 显示至屏幕上，并以并发操作

zkitefly · 2023-12-31T02:16:26Z

请将加载整合包文件的 Task 显示至屏幕上，并以并发操作

我觉得不太行，可能会影响操作流畅性

burningtnt · 2023-12-31T04:05:03Z

请将加载整合包文件的 Task 显示至屏幕上，并以并发操作

我觉得不太行，可能会影响操作流畅性

那就添加一个 Spinner，让用户感知到HMCL 正在加载

burningtnt · 2023-12-31T04:19:03Z

此外，建议把下面的仅文件名改为模组下载界面的 UI 风格，即，可以点进去查看详情

zkitefly · 2023-12-31T08:49:30Z

@CaveNightingale ping

CaveNightingale · 2023-12-31T09:21:42Z

@CaveNightingale ping

好

zkitefly · 2023-12-31T09:51:07Z

请将加载整合包文件的 Task 显示至屏幕上，并以并发操作

他看错了，所以不需要这样了

burningtnt · 2023-12-31T09:55:25Z

主要还是这一条：

此外，建议把下面的仅文件名改为模组下载界面的 UI 风格，即，可以点进去查看详情

和：

问题：若网络环境不佳，Curse 整合包下面的文件名显示是直接为空白，我觉得这个文件名获取可能会有点问题？

建议改成：每一个可选模组为一个 TwoLineListItem，可参考模组下载界面，如果失败，则显示“失败，点击重试”

zkitefly · 2023-12-31T10:02:07Z

我认为并不认同 改为模组下载界面的 UI 风格，就原来这个挺好的

如果有模组名称获取失败就重试几遍（5遍就够了），还是不行就直接贴一个 [加载失败，点击重试] 的一个小按钮

然后这个可选模组页面，加一个全选（全不选）按钮，然后加个 可选模组 标题我觉得就够了

burningtnt · 2023-12-31T10:05:25Z

我认为并不认同改为模组下载界面的 UI 风格，就原来这个挺好的

如果用户希望具体查看该可选模组的详细信息，那就需要改为我所提的这种风格了

类似这种 ↑

zkitefly · 2023-12-31T10:10:44Z

这样会不会增加复杂度啊

zkitefly · 2023-12-31T10:11:22Z

CaveNightingale · 2023-12-31T10:12:40Z

此外，建议把下面的仅文件名改为模组下载界面的 UI 风格，即，可以点进去查看详情

我认为并不认同改为模组下载界面的 UI 风格，就原来这个挺好的

如果用户希望具体查看该可选模组的详细信息，那就需要改为我所提的这种风格了类似这种 ↑

虽然但是，好像整合包文件里本来就没有模组的详细信息啊？

burningtnt · 2023-12-31T10:12:46Z

这样会不会增加复杂度啊

如果移动到单独界面呢？即：

将原来显示“加载可选模组”哪个地方删除
在安装左侧添加一个加载条，并在加载完毕后变为一个按钮，点击后进入新的界面展示模组列表并供用户勾选

用户只看一个模组的文件名，是无法考虑要还是不要这个模组的，确实需要展示模组详细信息

加油

burningtnt · 2023-12-31T10:16:40Z

By the way，请问你这里的 RT 是指？

RT，modrinth和curseforge整合包格式均支持声明可选模组/文件，而HMCL会直接将这些文件当作必选处理

hejiehao · 2023-12-31T10:18:13Z

此外，建议把下面的仅文件名改为模组下载界面的 UI 风格，即，可以点进去查看详情

我认为并不认同改为模组下载界面的 UI 风格，就原来这个挺好的

如果用户希望具体查看该可选模组的详细信息，那就需要改为我所提的这种风格了类似这种 ↑

虽然但是，好像整合包文件里本来就没有模组的详细信息啊？

Curse 整合包可以获取

CaveNightingale · 2023-12-31T10:18:50Z

虽然但是，好像整合包文件里本来就没有模组的详细信息啊？

你都拿到 Project ID 了，不就啥都有了

https://docs.modrinth.com/modpacks/format
但是隔壁Modrinth只给下载连接

CaveNightingale · 2023-12-31T10:19:06Z

By the way，请问你这里的 RT 是指？

RT，modrinth和curseforge整合包格式均支持声明可选模组/文件，而HMCL会直接将这些文件当作必选处理

如题

Minecraft269 · 2026-06-18T14:49:40Z

ping @CaveNightingale

敲打本PR作者日活非常低 :(

CaveNightingale · 2026-06-18T14:54:31Z

ping @CaveNightingale

敲打本PR作者日活非常低 :(

请问之前要求的渐变过渡是指什么效果？
我并没有看到有其他下一步的页面切换使用的是类似渐变的效果？

Minecraft269 · 2026-06-18T16:04:40Z

请问之前要求的渐变过渡是指什么效果？我并没有看到有其他下一步的页面切换使用的是类似渐变的效果？

不到啊问Glavo :(

Minecraft269 · 2026-06-21T05:20:14Z

CC @Glavo

Glavo · 2026-06-21T06:09:03Z

ContainerAnimations

Chen-Mengze · 2026-09-03T00:25:22Z

这个是目前搁置最久最老的 PR 了，我建议直接关掉重开，尤其是已经和主线冲突了，如果不是昨晚我手贱，估计类似更多#6805所要面对与解决的漏洞与隐患

CaveNightingale · 2026-09-03T02:22:36Z

这个是目前搁置最久最老的 PR 了，我建议直接关掉重开，尤其是已经和主线冲突了，如果不是昨晚我手贱，估计类似更多#6805所要面对与解决的漏洞与隐患

我基本上计划重新做了

CaveNightingale · 2026-09-03T06:06:20Z

我现在无法编译主线HMCL，此PR先搁置一下。

/home/cavenightingale/Projects/HMCL-main/HMCL/src/main/java/org/jackhuang/hmcl/ui/directory/GameDirectoryPage.java:264: 错误: 程序包MessageDialogPane不存在
            Controllers.confirm(i18n("game_directory.root"), i18n("message.warning"), MessageDialogPane.MessageType.WARNING, this::doSave, null);

3gf8jv4dv · 2026-09-03T07:24:53Z

我现在无法编译主线HMCL，此PR先搁置一下。

这个问题或许等 #6805 合并了就解决了

Minecraft269 · 2026-09-06T05:23:42Z

这个PR还活着吗? 要不先关了? Glavo目前正在重构整个HMCL底层可能会出现大量合并冲突导致工作流激增(甚至不如重写重开PR)

CaveNightingale · 2026-09-06T05:30:03Z

这个PR还活着吗? 要不先关了? Glavo目前正在重构整个HMCL底层可能会出现大量合并冲突导致工作流激增(甚至不如重写重开PR)

我的目前正在重写这个PR，你说Glavo重构正在重构整个HMCL底层，有没有RoadMap？我评估一下。

Minecraft269 · 2026-09-06T05:39:51Z

你说Glavo重构正在重构整个HMCL底层，有没有RoadMap？我评估一下。

重构比较慢但正在逐步重构 HMCL 的底层下一步动作等待Glavo的实际计划 (我猜可能要重构下载部分? (目前没明确的重构计划(自然RoadMap也是没有的(又体现到 #6602 的重要性了))))

目前已有6个直接在 Commit message 声明的重构(大)PR https://github.com/HMCL-dev/HMCL/issues?q=is%3Apr%20%E9%87%8D%E6%9E%84%20author%3AGlavo%20sort%3Acreated-desc

CaveNightingale · 2026-09-06T06:03:59Z

你说Glavo重构正在重构整个HMCL底层，有没有RoadMap？我评估一下。

重构比较慢但正在逐步重构 HMCL 的底层下一步动作等待Glavo的实际计划 (我猜可能要重构下载部分? (目前没明确的重构计划(自然RoadMap也是没有的(又体现到 #6602 的重要性了)))) 目前已有6个直接在 Commit message 声明的重构(大)PR https://github.com/HMCL-dev/HMCL/issues?q=is%3Apr%20%E9%87%8D%E6%9E%84%20author%3AGlavo%20sort%3Acreated-desc

感觉这都是一些局部即兴重构，我个人认为这个问题不是很大。即使过了这批，这类重构也会时常发生。这个PR也不止一次遇到一大堆合并冲突了。

github-actions

🤖 Static analysis found the following issues (max count 10):

CaveNightingale · 2026-09-06T07:16:54Z

ContainerAnimations

逻辑上，选择可选文件是选择文件名之后的下一步，使用向左滑动的动画和其他地方是一致的，用渐变怪怪的。

撑满高度的问题和没有可选文件显示可选文件的问题已经修改。

Glavo · 2026-09-08T12:52:20Z

@codex review

chatgpt-codex-connector · 2026-09-08T12:52:40Z

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-09-08T12:59:43.253818Z	`186d823`	Manual request

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 186d823c16

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

chatgpt-codex-connector · 2026-09-08T12:59:38Z

+    private void finishInstall(String name) {
+        controller.getSettings().put(MODPACK_NAME, name);
+        controller.getSettings().put(MODPACK_CHARSET, charset);
+        controller.getSettings().put(MODPACK_EXCLUDED_FILES, getExcludedFiles());


Preserve saved exclusions during updates

When updating a CurseForge or Modrinth instance that already has excluded.json, this page always starts with an empty excludedFiles set and stores that empty set here unless the user reselects every exclusion. The completion task then overwrites excluded.json with [] and downloads all previously excluded optional files, so a routine update silently discards the user's choices. Initialize the set from the target instance's persisted exclusions, or preserve the existing file until the selection is explicitly changed.

Useful? React with 👍 / 👎.

I think this is out of scope.

chatgpt-codex-connector · 2026-09-08T12:59:38Z

+            if (excludedFiles != null && excludedFiles.contains(file.key()))
                continue;


Remove already-installed files that become excluded

When an update excludes an optional file whose path and manifest identity are unchanged, this check only suppresses a new download. ModrinthInstallTask retains the existing file because it is still present in the new manifest, so the unchecked file remains installed; the CurseForge update path has the same behavior. Include exclusions in the update cleanup so the resulting instance actually reflects the selection.

Useful? React with 👍 / 👎.

Updating is not a part of this PR.

CaveNightingale · 2026-09-08T13:31:52Z

@Glavo 实现更新/更改整合包文件不是这个PR的目标，需要另外设计界面而且用户需求不多，我的建议是另PR处理。

CaveNightingale · 2026-09-08T13:31:57Z

@Glavo 实现更新/更改整合包文件不是这个PR的目标，需要另外设计界面而且用户需求不多，我的建议是另PR处理。

zkitefly mentioned this pull request Jul 6, 2023

添加 Modrinth 整合包可选择安装可选模组 #2255

Open

Minecraft269 mentioned this pull request Jun 27, 2026

优化鼠标光标 & 添加按钮等悬停效果 & 关闭动画时禁用水波纹动画 [Gemini] #5757

Open

github-actions Bot added the 500+ label Jul 17, 2026

add support for optional files at installation

1a2349e

CaveNightingale force-pushed the javafx branch from edc55e1 to 1a2349e Compare September 6, 2026 07:07

github-actions Bot added 1000+ and removed 500+ labels Sep 6, 2026

github-actions Bot reviewed Sep 6, 2026

View reviewed changes

clean imports

186d823

CaveNightingale requested review from 3gf8jv4dv, Glavo and zkitefly September 6, 2026 07:17

chatgpt-codex-connector Bot reviewed Sep 8, 2026

View reviewed changes

		if (excludedFiles != null && excludedFiles.contains(file.key()))
		continue;

Uh oh!

Conversation

CaveNightingale commented Oct 3, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

动机

描述

风险

非目标

Uh oh!

burningtnt commented May 23, 2023

Uh oh!

zkitefly commented Jul 29, 2023

Uh oh!

burningtnt commented Jul 29, 2023

Uh oh!

CaveNightingale commented Jul 29, 2023

Uh oh!

hejiehao commented Jul 30, 2023

Uh oh!

CaveNightingale commented Jul 30, 2023

Uh oh!

zkitefly commented Aug 7, 2023

Uh oh!

zkitefly commented Dec 31, 2023

功能请求

Uh oh!

zkitefly commented Dec 31, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Curse 整合包导入测试

Uh oh!

zkitefly commented Dec 31, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Modrinth 整合包导入测试

Uh oh!

zkitefly commented Dec 31, 2023

Modrinth 整合包安装测试

Uh oh!

burningtnt commented Dec 31, 2023

Uh oh!

zkitefly commented Dec 31, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

burningtnt commented Dec 31, 2023

Uh oh!

burningtnt commented Dec 31, 2023

Uh oh!

zkitefly commented Dec 31, 2023

Uh oh!

CaveNightingale commented Dec 31, 2023

Uh oh!

zkitefly commented Dec 31, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

burningtnt commented Dec 31, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

zkitefly commented Dec 31, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

burningtnt commented Dec 31, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

zkitefly commented Dec 31, 2023

Uh oh!

zkitefly commented Dec 31, 2023

Uh oh!

CaveNightingale commented Dec 31, 2023

Uh oh!

burningtnt commented Dec 31, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

burningtnt commented Dec 31, 2023

Uh oh!

hejiehao commented Dec 31, 2023

Uh oh!

CaveNightingale commented Dec 31, 2023

Uh oh!

CaveNightingale commented Dec 31, 2023

Uh oh!

Minecraft269 commented Jun 18, 2026

CaveNightingale commented Oct 3, 2022 •

edited

Loading

zkitefly commented Dec 31, 2023 •

edited

Loading

zkitefly commented Dec 31, 2023 •

edited

Loading

zkitefly commented Dec 31, 2023 •

edited

Loading

zkitefly commented Dec 31, 2023 •

edited

Loading

burningtnt commented Dec 31, 2023 •

edited

Loading

zkitefly commented Dec 31, 2023 •

edited

Loading

burningtnt commented Dec 31, 2023 •

edited

Loading

burningtnt commented Dec 31, 2023 •

edited

Loading

chatgpt-codex-connector Bot commented Sep 8, 2026 •

edited

Loading